Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[15.0][MIG] contract_payment_mode #811

Merged

Conversation

palkesh-aktivsoftware
Copy link
Contributor

Standard Migration

AngelMoya-Domatix and others added 30 commits March 28, 2022 16:57
* Copy partner payment mode to contracts when installing
* Filter payment modes for sales
* Adding tests
Co-Authored-By: Pedro M. Baeza <[email protected]>
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: contract-12.0/contract-12.0-contract_payment_mode
Translate-URL: https://translation.odoo-community.org/projects/contract-12-0/contract-12-0-contract_payment_mode/
Currently translated at 100.0% (2 of 2 strings)

Translation: contract-12.0/contract-12.0-contract_payment_mode
Translate-URL: https://translation.odoo-community.org/projects/contract-12-0/contract-12-0-contract_payment_mode/ca/
Currently translated at 100.0% (2 of 2 strings)

Translation: contract-12.0/contract-12.0-contract_payment_mode
Translate-URL: https://translation.odoo-community.org/projects/contract-12-0/contract-12-0-contract_payment_mode/pt_BR/
Currently translated at 100.0% (2 of 2 strings)

Translation: contract-12.0/contract-12.0-contract_payment_mode
Translate-URL: https://translation.odoo-community.org/projects/contract-12-0/contract-12-0-contract_payment_mode/zh_CN/
Currently translated at 100.0% (2 of 2 strings)

Translation: contract-12.0/contract-12.0-contract_payment_mode
Translate-URL: https://translation.odoo-community.org/projects/contract-12-0/contract-12-0-contract_payment_mode/de/
Currently translated at 100.0% (2 of 2 strings)

Translation: contract-12.0/contract-12.0-contract_payment_mode
Translate-URL: https://translation.odoo-community.org/projects/contract-12-0/contract-12-0-contract_payment_mode/pt/
Added view for supplier contracts, respecting the domain for payment_type.
Before was showing the inbound types in the supplier payment
Copy link

@misern2 misern2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link

@eduardgm eduardgm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functional review, LGTM!

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

if payment_mode:
contract.payment_mode_id = payment_mode.id
_logger.info("Setting payment mode: Done")
# with api.Environment.manage():
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do you remove the context manager ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, deprecated.

@palkesh-aktivsoftware Could you just remove the unneeded comment ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, I made this changes. Please review it.

@marylla
Copy link
Contributor

marylla commented Aug 12, 2022

Hi @rousseldenis Can you review again and merge?

@rousseldenis
Copy link
Contributor

/ocabot migration contract_payment_mode

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Aug 16, 2022
@OCA-git-bot OCA-git-bot mentioned this pull request Aug 16, 2022
18 tasks
@marylla
Copy link
Contributor

marylla commented Aug 18, 2022

@pedrobaeza @rousseldenis Who wants to merge this? It has a lot of approvals now. :)

@rousseldenis
Copy link
Contributor

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 15.0-ocabot-merge-pr-811-by-rousseldenis-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit ec0f712 into OCA:15.0 Aug 18, 2022
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 040b031. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.